Skip to content

Conversation

@prasad-cyble
Copy link
Contributor

@prasad-cyble prasad-cyble commented Jan 5, 2026

Bumped solution version to 3.0.3 in mainTemplate.json and updated all related analytical rule descriptions. Improved query parameter templates to use dynamic time window variables. Added new package file 3.0.3.zip. Refactored customDetails and incidentConfiguration property ordering for consistency across rules.

Required items, please complete

Change(s):

  • Changed api endpoint of the Data connector.
  • Minor descriptive changes to Analytic Rule and Playbook

Reason for Change(s):

  • As needed

Version Updated:

  • Yes

Testing Completed:

  • Yes

Bumped solution version to 3.0.3 in mainTemplate.json and updated all related analytical rule descriptions. Improved query parameter templates to use dynamic time window variables. Added new package file 3.0.3.zip. Refactored customDetails and incidentConfiguration property ordering for consistency across rules.
@prasad-cyble
Copy link
Contributor Author

Hello @v-maheshbh please take a look at the failing check

Renamed the 'HTMLURL' key to 'URL' in both Alerts_github.yaml and mainTemplate.json to ensure consistency in customDetails mapping. Updated the package archive to reflect these changes.
@prasad-cyble
Copy link
Contributor Author

Hello @v-maheshbh please take a look at the above PR.
Please review and merge

@prasad-cyble
Copy link
Contributor Author

Hello @v-maheshbh please run the workflow and provide your approval to the PR.

Thanks,
Prasad

Corrected typos in alert display name, updated API endpoint and increased page size in polling config, fixed JSON body formatting in playbook deployment, and improved post-deployment instructions. Also updated package files to reflect these changes.
@prasad-cyble
Copy link
Contributor Author

image

@prasad-cyble
Copy link
Contributor Author

hold

Updated API URIs in mainTemplate.json and azuredeploy.json to use the new /engine/api/v1/y/tpi/sentinel/alerts endpoints. Modified the alert update payload to use dynamic variables instead of hardcoded values. Also standardized postDeployment tag instructions for consistency.
@prasad-cyble
Copy link
Contributor Author

Hi Mahesh ready to review now

@prasad-cyble
Copy link
Contributor Author

@v-maheshbh can you please re run the checks

@v-maheshbh
Copy link
Contributor

v-maheshbh commented Jan 13, 2026

Hi @prasad-cyble
Please review the main template both inside the ZIP package and outside to ensure consistency and correctness.
image

Thanks!

@prasad-cyble
Copy link
Contributor Author

@v-maheshbh maintemplate outside the zip is correct.
Can you please compress the maintemplate and createui file for me and create a new zip.
As creating a zip in mac will include MACOS file in the zip.

Thanks in advance,
Prasad - Cyble

@prasad-cyble
Copy link
Contributor Author

Hi @v-maheshbh you can merge the PR.
as of now i cannot connect the data connector due to internal server issue. Connecting the connector will throw error

@v-atulyadav v-atulyadav merged commit 5286db0 into Azure:master Jan 13, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants